Skip to content

ASL Reference review#1758

Merged
Roman-Manevich merged 10 commits intomasterfrom
asl-reference-review
Mar 30, 2026
Merged

ASL Reference review#1758
Roman-Manevich merged 10 commits intomasterfrom
asl-reference-review

Conversation

@Roman-Manevich
Copy link
Copy Markdown
Collaborator

@Roman-Manevich Roman-Manevich commented Mar 23, 2026

This PR contains mainly fixes to issues found via an LLM reading.
Other changes:

  • Bumped document version to EAC1
  • Makefile used to depend on xxxLines.tex files, which were removed in a previous PR so I've added an explicit dependency on .tex files and asl.spec.
  • The reference used the old rule for annotate_stmt_STry, not the auto-generated one, for some reason. It now refers to the auto-generated one.
  • bot is now completely internal and does not appear in the reference nor in any spec file.
  • Function and relation notations are now consistent throughout the document (instead of having both f : A -> B and f(A) -> B we have just the latter form).
  • Cartesian product types now all use (A, B) where A and B are types to denote their Cartesian product type.
  • Converted Some and None to some and none throughout the document and spec (asl.spec and builtins.spec).
  • The concept of "configuration domain" is unused and was therefore removed.
  • Excluded runt. files from trailing whitespace check.

@Roman-Manevich Roman-Manevich changed the title Asl reference review ASL Reference review Mar 30, 2026
@Roman-Manevich Roman-Manevich requested a review from hrutvik March 30, 2026 12:03
@Roman-Manevich Roman-Manevich marked this pull request as ready for review March 30, 2026 12:03
@Roman-Manevich Roman-Manevich merged commit 608e73c into master Mar 30, 2026
5 checks passed
@Roman-Manevich Roman-Manevich deleted the asl-reference-review branch March 30, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants